Skip to content

Add coverage commenting#48

Merged
nx10 merged 11 commits into
mainfrom
maint/codecov
Feb 9, 2026
Merged

Add coverage commenting#48
nx10 merged 11 commits into
mainfrom
maint/codecov

Conversation

@kaitj
Copy link
Copy Markdown
Contributor

@kaitj kaitj commented Feb 9, 2026

@kaitj kaitj self-assigned this Feb 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 9, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/rbc
   __init__.py00100% 
src/rbc/cli
   __init__.py00100% 
   anatomical.py00100% 
   derivatives.py00100% 
   functional.py00100% 
   longitudinal.py00100% 
   qc.py00100% 
src/rbc/core
   __init__.py10100% 
   bids.py3620100% 
   common.py70100% 
   utils.py240100% 
src/rbc/core/anatomical
   __init__.py30100% 
   registration.py10460%25, 117, 132, 147
   segmentation.py10460%23, 45, 52, 63
src/rbc/core/functional
   __init__.py00100% 
src/rbc/core/longitudinal
   __init__.py00100% 
src/rbc/core/qc
   __init__.py00100% 
src/rbc/core/resources
   __init__.py80100% 
src/rbc/workflows
   anatomical.py15846%27–28, 30, 34, 37, 40, 45, 55
TOTAL4401696% 

Tests Skipped Failures Errors Time
42 0 💤 0 ❌ 0 🔥 47.664s ⏱️

@kaitj kaitj requested a review from nx10 February 9, 2026 20:39
@kaitj kaitj marked this pull request as ready for review February 9, 2026 20:40
@kaitj
Copy link
Copy Markdown
Contributor Author

kaitj commented Feb 9, 2026

This is finally working. Do want to include __init__.py in the coverage / should we hide those with 100% coverage?

Comment thread .github/workflows/test.yaml Outdated
Comment thread .github/workflows/test.yaml Outdated
@nx10 nx10 merged commit 8d3ef6a into main Feb 9, 2026
6 checks passed
@nx10 nx10 deleted the maint/codecov branch February 9, 2026 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants